Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds Portuguese (Brazilian) [solves #543] #599

Closed
wants to merge 13 commits into from

Conversation

dantop114
Copy link
Contributor

@dantop114 dantop114 commented Dec 3, 2019

Adding Portuguese (Brazilian) language.
Files structure follow the structure in PR #282.

Related issue

#543

samajammin and others added 13 commits November 7, 2019 08:10
Deploy Polish, Dutch & universal header IDs
Deploy /build page & persona re-org v1 for Ethereum Studio
Deploy Rust, Golang & .NET pages
Deploy Netlify Function rename
Deploy additional hero languages & new Slack invite link
Deploy Netlify Function build fix
Deploy /languages update to mention community translations
@samajammin
Copy link
Member

Thanks for the contribution @antodp!

Looking at those commits, they're all merge commits from dev branch into master. Did you branch off of master for this PR? That's what I suspect happened here. All PRs should branch off dev instead. You may want to try checking out the current dev, cherry-picking your commit, then opening a new PR. Let me know if you have questions or issues.

Copy link
Member

@samajammin samajammin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment above.

@dantop114 dantop114 closed this Dec 3, 2019
@dantop114 dantop114 deleted the pt-br-language branch December 3, 2019 17:23
@dantop114
Copy link
Contributor Author

dantop114 commented Dec 3, 2019

Hi @samajammin !
How can I do this? I think I need some help, because I synced the dev branch with master for mistake.

EDIT: Okay! Done in #604

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants